Skip to content

Feature/schwab login fixes#27

Merged
memoryfraction merged 14 commits into
mainfrom
feature/schwab-login-fixes
May 16, 2026
Merged

Feature/schwab login fixes#27
memoryfraction merged 14 commits into
mainfrom
feature/schwab-login-fixes

Conversation

@come150
Copy link
Copy Markdown
Collaborator

@come150 come150 commented May 12, 2026

完成嘉信的登陆和账户显示,并且增加如何加入这个账户说明

- 实现 ISchwabBrokerService 接口
- 支持账户信息查询
- 支持持仓信息获取
- 支持股票实时报价(单个和批量)
- 支持完整期权链数据获取
  - Call/Put 期权
  - 希腊字母(Delta, Gamma, Theta, Vega, Rho)
  - 隐含波动率
  - 未平仓合约和成交量
- 支持交易功能(下单、查询、取消)
- 支持市场状态查询
- OAuth 2.0 认证实现
- 完整的单元测试套件
- 详细的集成文档和使用示例
Copy link
Copy Markdown
Owner

@memoryfraction memoryfraction left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

请自查
1 删除非必要的文件,e.g. ps1
2 所有AI生成的md放进单独放进docs文件夹,如果没必要的可以直接删除
3 代码中的所有输出使用英文,防止命令行显示乱码
4 所有public级别的方法和属性加上中英文xml注释
5 所有代码强制符合SOLID原则

Comment thread ShowSchwabLoginDemo.ps1 Outdated
Clear-Host

Write-Host "╔" -NoNewline
Write-Host ("═" * 78) -NoNewline
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

删除非必要的PS1

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

好的

Comment thread .github/workflows/deploy-to-iis.yml Outdated
Comment thread -deploy.yml Outdated
Comment on lines +1 to +2
@page
@model DashboardModel
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

为什么需要web层? 必要性?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 增加单元测试,覆盖所有案例
2 删除web层

@memoryfraction memoryfraction merged commit 6419719 into main May 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants